Skip to content

Conversation

@jueunk617
Copy link
Collaborator

📌 개요

프론트엔드 요청사항에 맞춰 시그널링 서버의 DTO 및 에러 처리 로직을 수정하여 API 명세를 일치시켰습니다.

🔨 작업 내용

SDP 데이터 형식 변경

  • offer, answer 요청/응답 시 sdp 필드를 단순 String에서 프론트엔드와 약속한 Object 형식({"type": ..., "sdp": ...})으로 처리하기 위해 SdpData를 도입하고 관련 DTO를 모두 수정했습니다.

에러 응답 경로 및 형식 통일

  • @MessageExceptionHandler를 수정하여, 모든 예외 발생 시 프론트엔드와 약속한 WebRTCErrorResponse 형식으로 응답하도록 변경했습니다.
  • 모든 에러 응답은 요청자의 개인 채널인 /user/queue/webrtc로 전송되도록 수정하여, 성공/실패 응답 경로를 단일화했습니다.

컨트롤러 로직 개선

  • 인증 실패 시 errorHelper를 직접 호출하는 대신 CustomException을 던지도록 변경하여, 컨트롤러 내 모든 예외 처리 경로를 @MessageExceptionHandler로 통합했습니다.

테스트 코드 수정

  • 위 변경 사항에 맞춰 WebRTCSignalingControllerTest의 Mocking 및 검증 로직을 모두 수정했습니다.

🔗 관련 이슈

Closes #241

📝 참고 사항

  • 이번 수정으로 WebRTC 관련 모든 성공/실패 응답은 /user/queue/webrtc 단일 채널을 통해 전달됩니다.

✅ 체크리스트

  • 기능 동작 확인
  • 테스트 코드 작성
  • 문서/주석 추가 및 최신화

@github-actions github-actions bot changed the title Fix: 프론트 요청사항에 따른 WebRTC 시그널링 서버 수정 Fix: 프론트 요청사항에 따른 WebRTC 시그널링 서버 수정 (#241) Oct 12, 2025
@jueunk617 jueunk617 merged commit 856819e into dev Oct 12, 2025
3 checks passed
@github-actions github-actions bot deleted the Fix/241 branch October 12, 2025 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants